Skip to content

Fix a display bug which cannot update figure correctly #2144

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Dec 14, 2022

Conversation

Yichi-Lionel-Cheung
Copy link
Contributor

@Yichi-Lionel-Cheung Yichi-Lionel-Cheung commented Dec 8, 2022

When using notebook such as colab and local jupyter where is_ipython == True, the training procedure is not shown correctly. The figure is closed too fast to observe since clear_output(wait=True) is always activated.

Excanging display.display(plt.gcf()) and display.clear_output(wait=True) could fix the bug.

In addition, after training the model, the resulting figure contains all episodes shall be displaied.

cc @vmoens @nairbv

@netlify
Copy link

netlify bot commented Dec 8, 2022

Deploy Preview for pytorch-tutorials-preview ready!

Name Link
🔨 Latest commit 416c548
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-tutorials-preview/deploys/6399f74d3438170008590a7f
😎 Deploy Preview https://deploy-preview-2144--pytorch-tutorials-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@Yichi-Lionel-Cheung Yichi-Lionel-Cheung marked this pull request as ready for review December 8, 2022 13:59
@svekars svekars added the rl label Dec 8, 2022
@vmoens
Copy link
Contributor

vmoens commented Dec 8, 2022

LGTM
@svekars: can we wait to land this until we merge #2030 or #2145?
This refactoring has been waiting for a long time and it's a recurrent concern on the pytorch forum...
@Yichi-Lionel-Cheung it should not take long :) Hope that's ok

@Yichi-Lionel-Cheung
Copy link
Contributor Author

Sure!

@svekars
Copy link
Contributor

svekars commented Dec 8, 2022

@Yichi-Lionel-Cheung can you please resolve the conflict? Thanks!

@Yichi-Lionel-Cheung
Copy link
Contributor Author

Hi @svekars , conflict resolved

@svekars svekars merged commit 3c19831 into pytorch:master Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants